Skip to content

Conversation

oliverdr
Copy link
Member

@oliverdr oliverdr commented Sep 18, 2026

endring

retter feilene som stoppet quality checks etter #28. formatterer de 17 filene Prettier klaget på og oppdaterer testene fra ett væreksempel til dagens åtte byer.

værkort og detaljer bruker nå de eksisterende statuskomponentene. tomt API-svar får en beskjed og «prøv igjen», nye forsøk viser lasting, og knappen er deaktivert mens et tomt svar hentes på nytt. retter også ugyldig nesting i værkortenes dl/dt/dd og gjør den dempede teksten litt mørkere for å bestå kontrastsjekken.

fjerner én tom test uten assertions; den fungerende testen for feil og nytt forsøk beholdes. del av #13, #15 og #18. dette fullfører ikke sluttkontroll eller medstudentvurdering.

sjekker

  • npm run check: bestått. formatering, lint, genererte GraphQL-kontrakter, typecheck, migrasjonsmetadata, 28 Vitest-tester og begge bygg.
  • npm run test:e2e -- --workers=2: 25/25 bestått i Chromium, Firefox, WebKit og emulert iPhone stående/liggende. sjekker lasting, feil, tomme svar, tastatur-retry, ugyldige API-data, detaljer, horisontal overflow og axe-tilgjengelighet.
  • ingen genererte filer eller lockfiler er endret. ingen backend/database er startet.
  • testene bruker mockede værdata. fysiske mobiler og full deploy til VM er ikke bekreftet.
  • Actions-kjøring 68502: grønn på VM-runneren for commit 13e2b24. hele npm run check, deploy-/rollback-testen og alle 25 nettlesertestene bestod. deploy-jobben hoppes over i PR-er og kjører først ved vellykket push til main.

krav og KI

beholder plain CSS, egne komponenter, TanStack Query og live REST. codex hjalp med rettingene og kjørte kontrollene over. ingen nye docs-filer.

review

skal reviewes og merges manuelt av gruppa, ingen auto-merge. review- og quality-kravene er aktive igjen. etter merge til dev går publiseringen videre fra dev til main. den ekstra release-PR-en #30 er lukket.

@oliverdr oliverdr merged commit 526fb79 into dev Sep 18, 2026
2 checks passed
@oliverdr oliverdr deleted the fix/quality-checks branch September 18, 2026 10:05
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants